Skip to content

Conversation

@Sterling-Augustine
Copy link
Contributor

No description provided.

@Sterling-Augustine Sterling-Augustine merged commit 820a9b7 into llvm:main Apr 14, 2025
9 of 13 checks passed
@llvmbot
Copy link
Member

llvmbot commented Apr 14, 2025

@llvm/pr-subscribers-llvm-transforms

Author: None (Sterling-Augustine)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/135689.diff

1 Files Affected:

  • (modified) llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp (+1-1)
diff --git a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
index b2f495d445686..d0cb1c3ac590f 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
@@ -2414,7 +2414,7 @@ void VPlanTransforms::convertToConcreteRecipes(VPlan &Plan,
         VectorStep = Builder.createWidenCast(CastOp, VectorStep, IVTy);
       }
 
-      auto *ConstStep =
+      [[maybe_unused]] auto *ConstStep =
           ScalarStep->isLiveIn()
               ? dyn_cast<ConstantInt>(ScalarStep->getLiveInIRValue())
               : nullptr;

@llvmbot
Copy link
Member

llvmbot commented Apr 14, 2025

@llvm/pr-subscribers-vectorizers

Author: None (Sterling-Augustine)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/135689.diff

1 Files Affected:

  • (modified) llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp (+1-1)
diff --git a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
index b2f495d445686..d0cb1c3ac590f 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
@@ -2414,7 +2414,7 @@ void VPlanTransforms::convertToConcreteRecipes(VPlan &Plan,
         VectorStep = Builder.createWidenCast(CastOp, VectorStep, IVTy);
       }
 
-      auto *ConstStep =
+      [[maybe_unused]] auto *ConstStep =
           ScalarStep->isLiveIn()
               ? dyn_cast<ConstantInt>(ScalarStep->getLiveInIRValue())
               : nullptr;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants